chore: Use PanicHookInfo instead of PanicInfo#17766
Conversation
torokati44
left a comment
torokati44
left a comment
There was a problem hiding this comment.
Good to merge as soon as Rust 1.81 is out and CI passes. Would be nice to notify folks on Discord to rustup update after. Thanks!
|
I'd wait a few days for other distributions to catch up too, otherwise we'd lock up compilation of Ruffle for folks who do not use rustup. And it's not a critical PR, so IMO we can wait a bit |
torokati44
force-pushed
the
panic-info
branch
from
66d1186 to
658867a
Compare
Did you have any specific in mind, or just in general? |
|
Just in general, for instance Fedora takes a few days to deploy. You can see the current state of deployment here: https://repology.org/project/rust/versions |
|
Just noting that we may consider rolling back to 1.80.1 due to #17785. In that case this PR cannot be merged yet. |
Renamed in Rust 1.81.0, see https://releases.rs/docs/1.81.0/#compatibility-notes
torokati44
force-pushed
the
panic-info
branch
from
658867a to
537b0f8
Compare
|
Between all the other issues, distros have had plenty of time to catch up. Marked as ready for review. |
3 participants
Renamed in Rust 1.81.0, see https://releases.rs/docs/1.81.0/#compatibility-notes
CC @torokati44